Skip to content

Conversation

alexcrichton
Copy link
Member

I've seen these take up quite a bit of log space and I have the sneaking
suspicion that they're just making our test suite take longer (sometimes timing
out on 32-bit OSX now). In any case the backtraces haven't proven too useful,
unfortunately.

I've seen these take up quite a bit of log space and I have the sneaking
suspicion that they're just making our test suite take longer (sometimes timing
out on 32-bit OSX now). In any case the backtraces haven't proven too useful,
unfortunately.
@rust-highfive
Copy link
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@aturon
Copy link
Contributor

aturon commented Jan 26, 2017

@bors: r+

@bors
Copy link
Collaborator

bors commented Jan 26, 2017

📌 Commit 7095a48 has been approved by aturon

@alexcrichton
Copy link
Member Author

@bors: p=1

I think this is causing timeouts, so would love to test getting this in quickly to diagnose.

@bors
Copy link
Collaborator

bors commented Jan 27, 2017

⌛ Testing commit 7095a48 with merge 62a7258...

bors added a commit that referenced this pull request Jan 27, 2017
travis: Turn off core dumps on OSX

I've seen these take up quite a bit of log space and I have the sneaking
suspicion that they're just making our test suite take longer (sometimes timing
out on 32-bit OSX now). In any case the backtraces haven't proven too useful,
unfortunately.
@bors
Copy link
Collaborator

bors commented Jan 27, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: aturon
Pushing 62a7258 to master...

@bors bors merged commit 7095a48 into rust-lang:master Jan 27, 2017
@alexcrichton alexcrichton deleted the less-backtraces branch February 11, 2017 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants